Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

west.yml: update Zephyr to ca3310145f8f #8264

Closed
wants to merge 1 commit into from
Closed

west.yml: update Zephyr to ca3310145f8f #8264

wants to merge 1 commit into from

Conversation

iuliana-prodan
Copy link
Contributor

Contains 1131 commits, including the NXP hal update.

@iuliana-prodan
west.yml: update Zephyr to ca3310145f8f
7dff277 
Contains 1131 commits, including the NXP hal update.

Signed-off-by: Iuliana Prodan <[email protected]>
@iuliana-prodan iuliana-prodan mentioned this pull request Sep 26, 2023
Closed
lgirdwood
lgirdwood approved these changes Sep 26, 2023
View reviewed changes
Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@abonislawski @softwarecki @marcinszkudlinski good for you ?

@dbaluta
Copy link
Collaborator

dbaluta commented Sep 26, 2023

@lgirdwood can you help us merge this? Without it our internal CI cannot run and we are very close.to an internal milestone.

@tmleman
Copy link
Contributor

tmleman commented Sep 26, 2023

It has the same problems as mine.

Maybe with a smaller jump in zephyr revision it will work? #8268
I don't know how long it will take to resolve issue in our CI.

@iuliana-prodan
Copy link
Contributor Author

It has the same problems as mine.

Maybe with a smaller jump in zephyr revision it will work? #8268

We need this commit 492517b918 ("west.yml: Update NXP HAL SDK to 2.14") in Zephyr, so #8268 is good for our internal CI.

@dbaluta
Copy link
Collaborator

dbaluta commented Sep 27, 2023 •
edited
Loading

So, with a smaller jump like #8268 the CI will pass? We can use that so that we get our zephyr version updated?

@tmleman
Copy link
Contributor

tmleman commented Sep 27, 2023

So, with a smaller jump like #8268 the CI will pass? We can use that so that we get our zephyr version updated?

There are other problems but not as serious.

@iuliana-prodan
Copy link
Contributor Author

So, with a smaller jump like #8268 the CI will pass? We can use that so that we get our zephyr version updated?

There are other problems but not as serious.

I'll close this as soon as #8268 is merged.

@iuliana-prodan
Copy link
Contributor Author

iuliana-prodan commented Sep 28, 2023 •
edited
Loading

Zephyr was updated here: #8268

@iuliana-prodan iuliana-prodan deleted the zephyr_update_in_west_file branch October 13, 2023 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lgirdwood lgirdwood lgirdwood approved these changes

@dbaluta dbaluta dbaluta approved these changes

@plbossart plbossart Awaiting requested review from plbossart plbossart is a code owner

@mmaka1 mmaka1 Awaiting requested review from mmaka1 mmaka1 is a code owner

@lbetlej lbetlej Awaiting requested review from lbetlej lbetlej is a code owner

@kv2019i kv2019i Awaiting requested review from kv2019i kv2019i is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@iuliana-prodan @dbaluta @tmleman @lgirdwood